Open
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add fixed-position circular toggle button (top-right) with animated sun/moon SVG icons that crossfade on switch - Add :root[data-theme="light"] CSS variables (light backgrounds, dark text) so all existing elements inherit the correct palette automatically - Apply theme before first paint via inline <head> script reading localStorage to prevent flash of wrong theme - Persist user preference to localStorage; aria-label updates dynamically for screen reader accessibility - Use a .theme-transitioning class to temporarily apply 300ms color transitions across all elements during the switch Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add test_api.py with 18 tests covering POST /api/query, GET /api/courses,
and DELETE /api/session/{id} — including success, validation (422), and
error (500) cases
- Extend conftest.py with mock_rag_system, test_app, and client fixtures;
test_app mirrors app.py endpoints without static file mounting or real
RAGSystem init to keep tests isolated
- Add httpx dev dependency (required by FastAPI TestClient) and addopts
(-v --tb=short) to pytest config for cleaner output
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add black>=24.0.0 as a dev dependency with [tool.black] config (line-length 88, target py313) - Apply black formatting consistently across all 14 Python source and test files - Fix stale-sources bug in CourseSearchTool: reset last_sources=[] on empty results so sources don't leak between calls (resolves 1 failing test; all 35 now pass) - Add scripts/quality.sh with subcommands: format, check (CI-safe), test, all Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@claude Sample PR